Correct and compact service container doc #1635
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DominikaK
commented
Jun 2, 2022
DominikaK
commented
| Question | Answer |
|---|---|
| JIRA Ticket | n/a |
| Versions | all |
alongosz
requested changes
Jun 8, 2022
Comment on lines
+162
to
+164
| !!! tip | ||
|
|
||
| For a list of all service tags exposed by Symfony, see its [reference documentation]([[=symfony_doc=]]/reference/dic_tags.html). |
There was a problem hiding this comment.
For a follow up: we should have a similar page with Ibexa service tags.
alongosz
approved these changes
Jun 8, 2022
juskora
requested changes
Jun 9, 2022
docs/api/public_php_api.md
Outdated
| [[= product_name =]] uses the [Symfony service container]([[=symfony_doc=]]/service_container.html) for dependency resolution. | ||
|
|
||
| [Symfony dependency injection]([[=symfony_doc=]]/components/dependency_injection.html) ensures that any required services are available in your custom code | ||
| (for example controllers) when you inject them into the constructor. |
There was a problem hiding this comment.
Suggested change
| (for example controllers) when you inject them into the constructor. | |
| (for example, controllers) when you inject them into the constructor. |
| ## Field Types | ||
|
|
||
| The following tags used to register Field Type features in the [service container](https://doc.ibexa.co/en/latest/guide/service_container) have been renamed: | ||
| The following tags used to register Field Type features in the [service container](https://doc.ibexa.co/en/latest/guide/api/public_php_api.md#service-container) have been renamed: |
There was a problem hiding this comment.
This link and the others will be fixed when this PR is merged (because the releases section uses absolute links, they only work after merging).
| The `ezstudio.installer.studio_installer` service has been renamed to the FQCN-named | ||
| service `EzSystems\EzPlatformEnterpriseEditionInstallerBundle\Installer\Installer`. | ||
| Deprecated `ezplatform.ee.installer.class` [service container](https://doc.ibexa.co/en/latest/guide/service_container) parameter has been removed. | ||
| Deprecated `ezplatform.ee.installer.class` [service container](https://doc.ibexa.co/en/latest/guide/api/public_php_api.md#service-container) parameter has been removed. |
|
|
||
| The `eZ\Bundle\EzPublishCoreBundle\Controller` now extends `Symfony\Bundle\FrameworkBundle\Controller\AbstractController` instead of `Symfony\Bundle\FrameworkBundle\Controller\Controller` which has limited access to the [service container](https://doc.ibexa.co/en/latest/guide/service_container). | ||
| For details, see [Service Subscribers Locators](https://symfony.com/doc/5.0/service_container/service_subscribers_locators.html). | ||
| The `eZ\Bundle\EzPublishCoreBundle\Controller` now extends `Symfony\Bundle\FrameworkBundle\Controller\AbstractController` instead of `Symfony\Bundle\FrameworkBundle\Controller\Controller` which has limited access to the [service container](https://doc.ibexa.co/en/latest/guide/api/public_php_api.md#service-container). |
| The `eZ\Bundle\EzPublishCoreBundle\Controller` now extends `Symfony\Bundle\FrameworkBundle\Controller\AbstractController` instead of `Symfony\Bundle\FrameworkBundle\Controller\Controller` which has limited access to the [service container](https://doc.ibexa.co/en/latest/guide/service_container). | ||
| For details, see [Service Subscribers Locators](https://symfony.com/doc/5.0/service_container/service_subscribers_locators.html). | ||
| The `eZ\Bundle\EzPublishCoreBundle\Controller` now extends `Symfony\Bundle\FrameworkBundle\Controller\AbstractController` instead of `Symfony\Bundle\FrameworkBundle\Controller\Controller` which has limited access to the [service container](https://doc.ibexa.co/en/latest/guide/api/public_php_api.md#service-container). | ||
| For details, see [Service Subscribers Locators](https://symfony.com/doc/5.0/api/public_php_api.md#service-container/service_subscribers_locators.html). |
There was a problem hiding this comment.
Broken link (page not found on Symfony
| ### Symfony service container | ||
|
|
||
| The deprecated Symfony [service container](https://doc.ibexa.co/en/latest/guide/service_container) parameters ending with `.class` have been removed, services relying on them now have their classes defined explicitly. | ||
| The deprecated Symfony [service container](https://doc.ibexa.co/en/latest/guide/api/public_php_api.md#service-container) parameters ending with `.class` have been removed, services relying on them now have their classes defined explicitly. |
juskora
approved these changes
Jun 10, 2022
DominikaK
pushed a commit
that referenced
this pull request
Jun 10, 2022
DominikaK
pushed a commit
that referenced
this pull request
Jun 10, 2022
DominikaK
pushed a commit
that referenced
this pull request
Jun 10, 2022
DominikaK
pushed a commit
that referenced
this pull request
Jun 10, 2022
DominikaK
pushed a commit
that referenced
this pull request
Jun 10, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.